Codecov Report

❌ Patch coverage is 80.45977% with 17 lines in your changes missing coverage. Please review.
✅ Project coverage is 51.67%. Comparing base (84789e0) to head (62f1cff).

@@             Coverage Diff             @@
##           develop    #4411      +/-   ##
===========================================
+ Coverage    51.51%   51.67%   +0.15%     
===========================================
  Files          451      453       +2     
  Lines        11976    12014      +38     
  Branches      2066     2079      +13     
===========================================
+ Hits          6170     6208      +38     
+ Misses        5194     5191       -3     
- Partials       612      615       +3     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

PR Review

Summary

Adds server-driven featured/spotlight tool personalization to the tools dashboard. Introduces a personalized_featured_tool_order DB table (v27), a /resources/featured API endpoint, FeaturedToolsFlowProducer, and syncFeaturedTools() running in parallel with syncToolOrder(). Refactors ToolsPresenter to use these abstractions and extracts a shared toToolCardState() helper.

Checklist Findings

✅ Looks Good

• GodToolsRoomDatabase uses AutoMigration(from = 26, to = 27) — no manual migration needed
• storeFeaturedToolsFromSync and storePersonalizedToolOrderFromSync both @Transaction-wrapped with consistent reset + upsert pattern
• @Upsert used throughout for sync writes
• SyncToolPlaceholder shared by both sync paths
• FeaturedToolsFlowProducer correctly uses distinctUntilChanged() on the combined PERSONALIZATION flow
• syncFeaturedTools and syncToolOrder run concurrently via coroutineScope { launch; launch }
• normalizedCountry used consistently for API call, repository store, mutex key, and sync-time key in both syncFeaturedTools and syncToolOrder
• openToolDetails correctly extracted as a non-@Composable private fun called from event sinks
• toToolCardState() adds key(tool.code) to the spotlight branch that previously lacked it
• DAOs use Flow<T> for reactive queries, suspend fun for writes
• Test coverage comprehensive across FeaturedToolsFlowProducerTest, ToolSyncTasksTest, and ToolsRepositoryIT
• ktlint: PASS

⚠️ Minor Issues

(none)

❌ Must Fix

(none)

Overall Verdict

APPROVE — Clean, well-tested implementation following all project conventions.

🤖 Reviewed with Claude Code